Add option to install context-lens#41
Conversation
7c1db0e to
0f58eeb
Compare
0f58eeb to
c0fd5d7
Compare
d24d478 to
14c0eaf
Compare
No, not clear enough. It has to mention why it can't use spi |
bac7dd4 to
f498b5c
Compare
Is it good now? |
Outdated text?
pi speaks to CL?? |
f498b5c to
0f6447d
Compare
Removed that paragraph.
Rephrased. Should be clearer now. |
0f6447d to
8028e02
Compare
Is "It" the launcher? or context-lens? |
8028e02 to
4f7a624
Compare
Context-lens. Clarified this paragraph in commit message and PR description. |
4f7a624 to
f43c248
Compare
Added a --context-lens (-c) flag, that installs context-lens after installing Pi, and creates a launcher script "cpi" for it. context-lens is installed from git repository, with 2 patches applied: - Enable using PPQ.ai in Pi through MITM proxy. - When launching pi, add providers based on env. vars & auth.json. Launcher script launches context-lens with `pi` argument. Context-lens sets up pi (creates temp pi directory with modified settings), in particular to route its traffic through mitmproxy. Otherwise context-lens wouldn't be able to capture requests. Launcher script can't use `spi` because `pi` argument is not an executable name but a switch that tells context-lens to perform mentioned pi setup.
f43c248 to
60d1a70
Compare
|
Tried this today: So you're obviously missing to check if pnpm exists, and adding it as requirement in the same way you added mitmproxy. Maybe try to build context-lens with npm instead of pnpm? Maybe it works and then we don't need to add a new dependency. |
Building without pnpm can be done, but it requires executing commands in "build" scripts but with npm instead of pnpm and npx for other executables such as tsc. Which can get out of sync if the "build" script changes. |
Added a --context-lens (-c) flag, that installs context-lens after installing Pi, and creates a launcher script "cpi" for it.
context-lens is installed from git repository, with 2 patches applied:
Launcher script launches context-lens with
piargument. Context-lens sets up pi (creates temp pi directory withmodified settings), in particular to route its traffic through mitmproxy. Otherwise context-lens wouldn't be able to capture
requests. Launcher script can't use
spibecausepiargument is not an executable name but a switch that tells context-lens to perform mentioned pi setup.